backend/src/main/java/moadong/user/service/UserCommandService.java (1)

79-80: 토큰 개수 제한 및 정리 전략을 고려하세요.

현재 구현에서는 사용자당 리프레시 토큰 개수에 제한이 없습니다. 다음 사항들을 검토하시기 바랍니다:

1. 토큰 개수 제한: 사용자가 로그아웃 없이 계속 로그인하면 토큰이 무제한으로 누적될 수 있습니다. 최대 토큰 개수를 설정하고 오래된 토큰을 자동으로 제거하는 정책을 고려하세요 (예: 최대 5개 기기).

2. 만료된 토큰 정리: 만료된 토큰이 DB에 계속 남아있습니다. 주기적인 정리 작업(scheduled job)이나 로그인 시 만료 토큰 제거 로직을 추가하는 것이 좋습니다.

3. 기기 식별: 현재는 어떤 토큰이 어떤 기기에서 사용되는지 추적할 수 없습니다. 향후 "다른 기기에서 로그아웃" 같은 기능을 위해 기기 정보(User-Agent, 기기명 등)를 RefreshToken에 포함하는 것을 고려하세요.

backend/src/main/java/moadong/user/entity/User.java (2)

107-118: 토큰을 찾지 못한 경우의 처리를 개선하세요.

replaceRefreshToken 메서드가 oldToken을 찾지 못하면 아무 작업 없이 조용히 반환됩니다. 이는 토큰 교체가 예상되지만 실제로 발생하지 않는 버그를 숨길 수 있습니다.

다음 중 하나를 고려하세요:

• 토큰을 찾지 못한 경우 예외를 던지기
• 경고 로그 남기기
• 반환 타입을 boolean으로 변경하여 성공 여부를 알리기

🔎 예외 처리 추가 제안

 public void replaceRefreshToken(String oldToken, RefreshToken newToken) {
     if (this.refreshTokens == null) {
         this.refreshTokens = new ArrayList<>();
-        return;
+        throw new IllegalStateException("토큰 교체 실패: 리프레시 토큰 목록이 비어있습니다.");
     }
     for (int i = 0; i < this.refreshTokens.size(); i++) {
         if (this.refreshTokens.get(i).getToken().equals(oldToken)) {
             this.refreshTokens.set(i, newToken);
             return;
         }
     }
+    throw new IllegalArgumentException("토큰 교체 실패: 기존 토큰을 찾을 수 없습니다.");
 }

---

127-133: 전체 토큰 제거 메서드의 null 처리를 단순화할 수 있습니다.

현재 구현은 refreshTokens가 null일 때 새 ArrayList를 초기화한 후 바로 반환합니다(129-130행). null인 경우 초기화 없이 바로 반환하거나, null이 아닐 때만 clear를 호출하는 것이 더 명확합니다.

🔎 단순화 제안

 public void removeAllRefreshTokens() {
-    if (this.refreshTokens == null) {
-        this.refreshTokens = new ArrayList<>();
-        return;
-    }
-    this.refreshTokens.clear();
+    if (this.refreshTokens != null) {
+        this.refreshTokens.clear();
+    }
 }

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.